Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 30, 2025

Fix spelling: change "ack'd" to "acked"

Description

Addresses feedback from PR #148 code review. Changes the abbreviation "ack'd" to "acked" for consistency with standard spelling conventions in technical documentation.

  • ModeManager.fpp:181 - Event format string
  • docs/sdd.md:476 - Documentation table

Related Issues/Tickets

How Has This Been Tested?

Documentation-only change (spelling fix in format strings and markdown).

  • Unit tests
  • Integration tests
  • Z Tests
  • Manual testing (describe steps)

Screenshots / Recordings (if applicable)

N/A - text-only change

Checklist

  • Written detailed sdd with requirements, channels, ports, commands, telemetry defined and correctly formatted and spelled
  • Have written relevant integration tests and have documented them in the sdd
  • Have done a code review with
  • Have tested this PR on every supported board with correct board definitions

Further Notes / Considerations

Trivial spelling fix. No functional changes.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copy link
Contributor Author

Copilot AI commented Nov 30, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Apply changes based on feedback for hibernation mode PR Fix spelling: change "ack'd" to "acked" for consistency Nov 30, 2025
Copilot AI requested a review from yudataguy November 30, 2025 04:52
Copilot finished work on behalf of yudataguy November 30, 2025 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants